Skip to content

[limen GEN-a-organvm-call-function--ontological-test-coverage-0620] Raise test coverage in a-organvm/call-function--ontological#13

Merged
4444J99 merged 1 commit into
mainfrom
limen/gen-a-organvm-call-function--ontological-test-coverage-0620-d74c
Jul 19, 2026
Merged

[limen GEN-a-organvm-call-function--ontological-test-coverage-0620] Raise test coverage in a-organvm/call-function--ontological#13
4444J99 merged 1 commit into
mainfrom
limen/gen-a-organvm-call-function--ontological-test-coverage-0620-d74c

Conversation

@4444J99

@4444J99 4444J99 commented Jul 1, 2026

Copy link
Copy Markdown
Member

Autonomous limen dispatch of task GEN-a-organvm-call-function--ontological-test-coverage-0620.

Find the largest source module in a-organvm/call-function--ontological with little or no test coverage and add a focused, PASSING test suite for it. Run the repo's own test command and confirm green. No placeholder tests. [auto-generated 2026-06-20 to keep the stream endless]

Produced in an isolated worktree off origin — review before merge.

limen task GEN-a-organvm-call-function--ontological-test-coverage-0620
@4444J99
4444J99 enabled auto-merge (squash) July 1, 2026 06:39

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a3c4505cbb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -0,0 +1,163 @@
"""Tests for the FUNCTIONcalled naming validator."""

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Put coverage tests where CI runs them

This new test module is added under tests/, but the repository's current validation workflow runs python -m pytest tools/ -v in .github/workflows/validate.yml line 42, and the contributor docs use the same tools/ test command. As a result, these coverage tests are skipped in CI and by the documented test command, so regressions in tools/validate_naming.py still won't be caught unless someone manually runs plain pytest; either move the tests under tools/ or update the workflow/docs command to include tests/.

Useful? React with 👍 / 👎.

@4444J99
4444J99 merged commit d7cf487 into main Jul 19, 2026
4 checks passed
@4444J99
4444J99 deleted the limen/gen-a-organvm-call-function--ontological-test-coverage-0620-d74c branch July 19, 2026 10:47
@4444J99

4444J99 commented Jul 19, 2026

Copy link
Copy Markdown
Member Author

Disposition: merged:d7cf48772e0d3307b060cc5378f0f2886641387f.

Verified this as substantive coverage before merging: the PR adds tests/test_validate_naming.py with focused naming-validator tests, its visible checks were green, and local validation on the PR passed (python3 -m pytest -q = 208 passed; make validate; make validate-naming). The only merge blocker was the known structural validate-dependencies context mismatch, so I used the allowed admin squash path. After merging the safe PR stack, default branch still validates green: 268 pytest tests plus make validate and make validate-naming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant